Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 59.30%. Comparing base (244fb6a) to head (6da36df).
Report is 54 commits behind head on main.

@@            Coverage Diff             @@
##             main     #366      +/-   ##
==========================================
+ Coverage   56.86%   59.30%   +2.43%     
==========================================
  Files          16       16              
  Lines         728      747      +19     
  Branches      113      162      +49     
==========================================
+ Hits          414      443      +29     
+ Misses        303      293      -10     
  Partials       11       11              

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

Thanks for the PR. I would love to have further investigation before merging this (otherwise ofetch ends up with lots of old browser compatibility fixes)

Do you mind to make a minimal testing environment (html+js) and share the browser version/platform required to verify.

Dear @pi0，I apologize, but I am unable to provide a reproducible test case.

On one hand, it only occurs in the QQ Browser on iOS, and I am not very clear about the WebKit version being used (my phone is running iOS 17). 

Also, I am not very familiar with testing on old browsers, so I apologize for not being able to provide a shareable test environment. 😭

Okay, no worries. I will try to assemble a reproduction env once could but it is essential before we can move it forward I hope it is reasonable.

Okay, no worries. I will try to assemble a reproduction env once could but it is essential before we can move it forward I hope it is reasonable.

Thank you 🥰

If someone doesn't have high stability requirements, they can use this module → markthree/nuxt-fix-ofetch

Seems to be fixed in the latest QQ browser @markthree can you confirm?

#294 (comment)

Seems to be fixed in the latest QQ browser @markthree can you confirm?

#294 (comment)

Yes, the problem still exists, it's a problem with ofetch, and it can be reproduced when using it solely

From @xjccc comment:

now..
QQ version: 15.1.6.5925
UC version: 16.5.5.2341
works fine

But you have issue with these versions and ofetch?

From @xjccc comment:

now..
QQ version: 15.1.6.5925
UC version: 16.5.5.2341
works fine

But you have issue with these versions and ofetch?

yes, QQ Browser version: 15.1.7

Thanks to the onResponse option, I'm now using it to hack in my production environment, and so far I've found no problems! 🤣

ref → nuxt-fix-ofetch/src/runtime/00.fetch.client.ts#L52-#L86

I upgrade my QQ Browser version: 15.1.7.5928

iOS version: 17.5.1

• Operating System: Darwin
• Node Version: v20.10.0
• Nuxt Version: 3.12.2
• CLI Version: 3.12.0
• Nitro Version: 2.9.6
• Package Manager: pnpm@8.15.4

my web site works fine..

Hi dear @markthree Again sorry this took long.

We have merged a relative fix in #398 that passes _bodyInit directly as _data.

I think we might add an improvement to check if body is blob, use .stream()

Hi dear @markthree Again sorry this took long.

We have merged a relative fix in #398 that passes _bodyInit directly as _data.

I think we might add an improvement to check if body is blob, use .stream()

Thanks @pi0, looks great, ofetch is much easier to use.